Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Histogram distribution text and E(x) formula #28

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

rakhimov
Copy link
Member

The Histogram distribution is reviewed to correct the formula and the text in general.
Confusing use of x and E is avoided;
instead, conventional b for boundaries and w for weights are used in the formulas and text.

Fixes #27

@rakhimov
Copy link
Member Author

The html result can be viewed from artifacts.

The confusing E_i is swapped for w_i for weights.
The x is confusing in this context
due to conventional use of x as the random variable
(result of sampling).
The range is fixed to be right-exclusive
as customary in other distribution sampling.
The two steps were confused in the sampling description.
The steps are rearranged and clarified.
The =>Histogram formulas are fixed.
The text looks very confusing and out-of-place,
being a sign that these distributions should be provided separately.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant